Skip to content

Clarify trigger semantics for pipeline-errors 'threshold' and 'active'#1494

Open
earies wants to merge 1 commit into
openconfig:masterfrom
earies:pipeline-counters-threshold
Open

Clarify trigger semantics for pipeline-errors 'threshold' and 'active'#1494
earies wants to merge 1 commit into
openconfig:masterfrom
earies:pipeline-counters-threshold

Conversation

@earies
Copy link
Copy Markdown
Contributor

@earies earies commented May 13, 2026

  • (M) release/models/platform/openconfig-platform-pipeline-counters.yang
    • Clarify trigger semantics for 'threshold' and 'active'
    • Increment to version 0.5.2

Change Scope

Clarify the trigger semantics shared by pipeline-errors-common threshold and
active leafs. Documentation only change, no schema changes.

Platform Implementations

N/A

Tree View

No tree updates were made as part of this patchset.

  * (M) release/models/platform/openconfig-platform-pipeline-counters.yang
    - Clarify trigger semantics for 'threshold' and 'active'
    - Increment to version 0.5.2
@earies earies requested a review from a team as a code owner May 13, 2026 15:44
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the openconfig-platform-pipeline-counters YANG model to version 0.5.2. It adds a new revision entry and provides clearer descriptions for the threshold and active leaves regarding their trigger and transition semantics. A review comment correctly identifies that the revision date is set to a future date and should be corrected.

Comment thread release/models/platform/openconfig-platform-pipeline-counters.yang
@dplore dplore moved this to Ready to discuss in OC Operator Review May 15, 2026
@dplore
Copy link
Copy Markdown
Member

dplore commented May 15, 2026

/gcbrun

@OpenConfigBot
Copy link
Copy Markdown

No major YANG version changes in commit 9987b23

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Ready to discuss

Development

Successfully merging this pull request may close these issues.

3 participants